Skip to content

[WIP] Add known_hosts file syncing#31

Closed
Copilot wants to merge 3 commits intomainfrom
copilot/fix-30
Closed

[WIP] Add known_hosts file syncing#31
Copilot wants to merge 3 commits intomainfrom
copilot/fix-30

Conversation

Copy link
Contributor

Copilot AI commented May 22, 2025

  • Update the SshConfig model to include a KnownHosts field
  • Update the data transfer functions to handle known_hosts file
  • Update tests to include the new known_hosts functionality
  • Update README.md to document the new feature

Fixes #30.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: therealpaulgg <21159486+therealpaulgg@users.noreply.github.com>
@therealpaulgg
Copy link
Owner

@copilot poke

@therealpaulgg
Copy link
Owner

@copilot it seems you died, can you continue?

Co-authored-by: therealpaulgg <21159486+therealpaulgg@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented May 23, 2025

@copilot it seems you died, can you continue?

I've completed the implementation to add known_hosts file syncing. Changes include:

  1. Added a KnownHosts field to the SshConfig model
  2. Created a custom ServerSshConfigDto to handle the known_hosts data
  3. Updated the data transfer functions to include known_hosts field
  4. Updated the repository layer SQL queries to include known_hosts
  5. Fixed tests to work with the new field
  6. Updated the README with documentation

All tests are passing. The implementation is in commit b7708df.

@therealpaulgg
Copy link
Owner

should probably update the main library to support this before trying to work on ssh-sync-server as copilots getting confused

@therealpaulgg
Copy link
Owner

This implementation is a mess, closing until the ssh-sync client one is completed. then maybe we can try again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add known_hosts file syncing

2 participants